Added updateOnly option to createShortcut command #1368
Conversation
Co-Authored-By: SamvelRaja <[email protected]>
|
@kutu Changes looks fine? |
|
@SamvelRaja yes, i have tested it, and it works as expected, thx |
|
@kutu will it be merged? |
|
How do I know, I'm nobody in this project |
shiftkey
added
the
unmergeable
There was a problem hiding this comment.
Thank you for making this change. I hope that this can get merged in because I have need for a similar feature in a project I'm working on. It looks valid to me, but I am still new to this code so I'll defer to the maintainers.
Looking over the contributing docs, it looks like this needs to be moved to "develop" instead of master, and get merged up. I'm happy to re-create this PR myself if needed to keep this alive.
Adding @shiftkey who marked this as unmergable. What can I do to pick up this work and get it moving?
Editing to add that I have created #1674 to recreate this. @SamvelRaja thanks for creating this in the first place and I hope you don't mind it getting bumped forward.
The createShorcutForExecutable method has the ability to do differentiation between whether it is a updateOnly action or a create Action but this ability is not exposed via command line.
This PR enables the feature, feel free to decline it if it is not aligning with the roadmap.